-
Notifications
You must be signed in to change notification settings - Fork 67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add pattern synonyms for IsSequence
#250
base: master
Are you sure you want to change the base?
Conversation
@snoyberg I am seeing discrepancies in the versioning of classy-prelude of using 3 and 4 numbering schemes. What consistent versioning scheme did you intend for this to be? |
@vigress8 Can you check your haddock comment for Empty and Cons? I am unsure if they will render correctly for Cons. |
I try to stick to PVP versioning, no strong preference if a fourth number is included or not, your call. |
@BebeSparkelSparkel thanks, fixed that. Should be fine now |
@vigress8 Could you look at the build log https://github.com/snoyberg/mono-traversable/actions/runs/12178111855/job/34073973421?pr=250 It seems to be indicating that the cabal and package.yaml files are out of sync. |
@BebeSparkelSparkel could you approve the workflow run? |
@vigress8 I tried running the workflow but brew has dropped llvm@13 because it's a new year. I have updated the workflow matrix to exclude the latest macos and the old version of stack but when I tried to rerun on your changes it did not exclude the failing workflows. Can you rebase onto snoyberg:master and add a inconsequential change in your code to trigger github allow another action run. |
b0ba78d
to
2354ceb
Compare
@BebeSparkelSparkel done |
Closes #241.
CC @BebeSparkelSparkel @gregwebs